Skip to content

Sdk11.1 cyclictest and misc cleanups#352

Merged
praneethbajjuri merged 8 commits into
TexasInstruments:masterfrom
bryanbrattlof:sdk11.1-cyclictest
Oct 16, 2025
Merged

Sdk11.1 cyclictest and misc cleanups#352
praneethbajjuri merged 8 commits into
TexasInstruments:masterfrom
bryanbrattlof:sdk11.1-cyclictest

Conversation

@bryanbrattlof

Copy link
Copy Markdown
Collaborator

No description provided.

@praneethbajjuri

Copy link
Copy Markdown
Collaborator

Since there are multiple platform variants for AM62x.
Please document clearly on which board the performance numbers are captured on.
Under
"The latencies observed with this SDK are summarized below:"
may be update as
"The latencies observed with this default yocto SDK image on https://www.ti.com/tool/SK-AM62B-P1 and are summarized below:"

Comment thread source/devices/AM62AX/linux/RT_Linux_Performance_Guide.rst Outdated
Comment thread source/devices/AM62AX/linux/RT_Linux_Performance_Guide.rst Outdated
Comment thread source/devices/AM62PX/linux/RT_Linux_Performance_Guide.rst Outdated
@bryanbrattlof

Copy link
Copy Markdown
Collaborator Author

Yeah I agree @praneethbajjuri . I've updated that sentence to be a little more specific.

@StaticRocket

Copy link
Copy Markdown
Member

Could you address the vale comments?

Comment thread source/devices/AM62AX/linux/RT_Linux_Performance_Guide.rst Outdated
Comment thread source/devices/AM62PX/linux/RT_Linux_Performance_Guide.rst Outdated

@praneethbajjuri praneethbajjuri left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets hold off on integration of this . Till the investigation completes on why latency numbers changed.
Might need to expand this PR to include the rootcause and workaround details to get better perf numbers.

@cshilwant cshilwant marked this pull request as draft July 11, 2025 16:19
@cshilwant

Copy link
Copy Markdown
Member

moved it to draft based on #352 (review)

Comment thread source/devices/AM62AX/linux/RT_Linux_Performance_Guide.rst
@bryanbrattlof bryanbrattlof marked this pull request as ready for review October 2, 2025 13:35
@github-actions

github-actions Bot commented Oct 2, 2025

Copy link
Copy Markdown

New warnings found with rstcheck:

source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:584: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:595: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:607: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:619: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:652: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:665: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:680: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:694: (SEVERE/4) Title level inconsistent:

@praneethbajjuri

Copy link
Copy Markdown
Collaborator

@dao-qiu please review

praneethbajjuri
praneethbajjuri previously approved these changes Oct 2, 2025
@praneethbajjuri

Copy link
Copy Markdown
Collaborator

@bryanbrattlof can you update in commitmsg and histogram as well on what is the exact tag used for a given device to get this numbers.
Example:
AM62A 11.1 is RC7
AM62P/62/64/ 11.1 is RC5
AM62L 11.1 and AM62P 11.1.1 is RC16 ( Wait till this PR is merged #459 )

@StaticRocket

Copy link
Copy Markdown
Member

@praneethbajjuri praneethbajjuri changed the title Sdk11.1 cyclictest Sdk11.1 cyclictest and misc cleanups Oct 13, 2025

@StaticRocket StaticRocket left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Those rstcheck errors are valid. Seems like the template needs to be adjusted.

Comment thread source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst Outdated
Comment thread source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst Outdated
The lack of sections in this page makes cross linking difficult. Convert
the bullet points and rubrics to proper sections.

Signed-off-by: Bryan Brattlof <bb@ti.com>
@github-actions

Copy link
Copy Markdown

New warnings found with rstcheck:

source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:598: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:610: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:622: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:655: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:668: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:683: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:697: (SEVERE/4) Title level inconsistent:

StaticRocket
StaticRocket previously approved these changes Oct 14, 2025
Comment thread source/linux/Foundational_Components_OPTEE.rst Outdated
@dao-qiu

dao-qiu commented Oct 15, 2025

Copy link
Copy Markdown

@bryanbrattlof can you update in commitmsg and histogram as well on what is the exact tag used for a given device to get this numbers. Example: AM62A 11.1 is RC7 AM62P/62/64/ 11.1 is RC5 AM62L 11.1 and AM62P 11.1.1 is RC16 ( Wait till this PR is merged #459 )

@bryanbrattlof Is there any feedback on the exact RCs used for the cyclictest results?

From Bryan: I updated all the graphs to include which release candidate was used by each board

Comment thread source/devices/AM62AX/linux/RT_Linux_Performance_Guide.rst
In certain highly specific use-cases the TRNG drivers in OPTEE could
have unwanted affects to the overall system if the application cares
more about deterministic latency over entropy.

Provide a section to document how to easily switch from the true RNG
drivers to pseudo RNG drivers.

Signed-off-by: Bryan Brattlof <bb@ti.com>
Newer silicon revisions can now integrate OP-TEE into their builds. Add
the OP-TEE foundation component back into the AM62LX build

Signed-off-by: Bryan Brattlof <bb@ti.com>
Update the real time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <bb@ti.com>
Update the real-time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <bb@ti.com>
Update the real-time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <bb@ti.com>
Update the real-time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <bb@ti.com>
Update the real-time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <bb@ti.com>
@github-actions

Copy link
Copy Markdown

New warnings found with rstcheck:

source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:598: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:610: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:622: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:655: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:668: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:683: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:697: (SEVERE/4) Title level inconsistent:

@bryanbrattlof

Copy link
Copy Markdown
Collaborator Author

@bryanbrattlof can you update in commitmsg and histogram as well on what is the exact tag used for a given device to get this numbers. Example: AM62A 11.1 is RC7 AM62P/62/64/ 11.1 is RC5 AM62L 11.1 and AM62P 11.1.1 is RC16 ( Wait till this PR is merged #459 )

@bryanbrattlof Is there any feedback on the exact RCs used for the cyclictest results?

Ah sorry (I didn't see a reponse dialog and skipped it) yep I've updated the graphs to indicate which release candidate was used for each

@cshilwant cshilwant requested a review from r-vignesh October 16, 2025 15:57
@praneethbajjuri praneethbajjuri merged commit ae7ac37 into TexasInstruments:master Oct 16, 2025
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants